Skip to content

Document ID token authentication for service accounts (#1350 - 8)#1875

Merged
koverholt merged 10 commits into
mainfrom
Token-id-update-1350---8
Jul 6, 2026
Merged

Document ID token authentication for service accounts (#1350 - 8)#1875
koverholt merged 10 commits into
mainfrom
Token-id-update-1350---8

Conversation

@zyantw

@zyantw zyantw commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Documents ID token support for service-account authentication, needed when an agent calls IAM-restricted services such as private Cloud Run or Cloud Functions. Adds a "Use ID token" section under "Configure tools with authentication" in the Tool Authentication guide, with an end-to-end OpenAPIToolset example, the ServiceAccount config parameters, and troubleshooting tips.

Rendered preview: https://deploy-preview-1875--adk-docs-preview.netlify.app/tools-custom/authentication/#use-id-token

Original Issue: #1350 (see item 8)

Original adk-bot PR: #1365. This PR supersedes #1365 (adk-bot's auto-generated draft, which used a helper signature that doesn't exist); it uses the actual ServiceAccount API instead, so #1365 can be closed.

@zyantw zyantw requested a review from joefernandez June 25, 2026 22:58
@netlify

netlify Bot commented Jun 25, 2026

Copy link
Copy Markdown

Deploy Preview for adk-docs-preview ready!

Name Link
🔨 Latest commit 8a9e1b6
🔍 Latest deploy log https://app.netlify.com/projects/adk-docs-preview/deploys/6a4bdbfab9542f0008cef4ea
😎 Deploy Preview https://deploy-preview-1875--adk-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@joefernandez joefernandez left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some teaching flow issues and style/mechanical issues to look at, here

Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md
Comment thread docs/tools-custom/authentication.md
@koverholt

koverholt commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

FYI the new failing link checker CI is due to a different HTTP status code coming from one of Atlassian's websites, unrelated to this PR. I'll fix that in a separate PR shortly.

@koverholt koverholt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates! I verified the ServiceAccount API against adk-python and the core is correct: the params, the audience-required-when-use_id_token validation, and the Cloud Run/Functions purpose all check out.

I left inline comments for the fixes needed before merge. Once those are in I can re-review.

Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md
Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md Outdated
Comment thread docs/tools-custom/authentication.md Outdated
@koverholt koverholt changed the title 1350 - 8 Token ID update Document ID token authentication for service accounts (#1350 - 8) Jul 6, 2026

@koverholt koverholt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the edits and working through all of the comments. The updated content LGTM, merging.

@koverholt koverholt merged commit 29a1d4d into main Jul 6, 2026
10 of 12 checks passed
@koverholt koverholt deleted the Token-id-update-1350---8 branch July 6, 2026 16:47
@zyantw zyantw self-assigned this Jul 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants